Skip to content

Conversation

@IlyaEp
Copy link
Contributor

@IlyaEp IlyaEp commented Nov 17, 2025

No description provided.

@IlyaEp IlyaEp requested a review from a team as a code owner November 17, 2025 08:04
@addaleax
Copy link
Collaborator

@IlyaEp Fyi, this is failing in CI:

 [2025/11/17 11:17:12.217] com.mongodb.<REDACTED:notary_signing_key_name>.CursorTest > classMethod FAILED
 [2025/11/17 11:17:12.217]     java.lang.UnsupportedClassVersionError: org/graalvm/polyglot/Context has been compiled by a more recent version of the Java Runtime (class file version 61.0), this version of the Java Runtime only recognizes class file versions up to 60.0

I'm guessing that means we need a more recent Java version in CI?

@IlyaEp
Copy link
Contributor Author

IlyaEp commented Nov 17, 2025

@addaleax I guess yes

@addaleax addaleax changed the title Update org.graalvm.polyglot and org.graalvm.js to 25.0.1 feat(java-shell)!: pdate org.graalvm.polyglot and org.graalvm.js to 25.0.1 Nov 24, 2025
@addaleax addaleax changed the title feat(java-shell)!: pdate org.graalvm.polyglot and org.graalvm.js to 25.0.1 feat(java-shell)!: update org.graalvm.polyglot and org.graalvm.js to 25.0.1 Nov 24, 2025
@addaleax addaleax added the no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) label Nov 24, 2025
@@ -0,0 +1,3 @@
LongResult: 2147483649
LongResult: 1281104301025214464
LongResult: 1281104301025214467
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@IlyaEp Allowing this result makes the tests pass, I assume this is coming from the graalvm update

@addaleax addaleax merged commit 382a11f into mongodb-js:main Nov 24, 2025
153 of 155 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants